image: add gtk_image_get_preferred_size()

Add a function to query the preferred size of the GtkImage, and use it
in the get_preferred_width/height implementations.

treeview: Ensure selection's changed signal

There was a corner case where the changed signal was not emitted.
If rows were built like this:
  1    (not selected)
  + 2  (selected)
  + 3  (not selected)
And row 1 was removed, no signal would be emitted.

I like it when writing tests actually finds bugs that have been around
since 2003 - introduced by 4a03ea2334bde7d2e15d0933c5a78f5a2338a854
actually. :)

treeview: Don't try to optimize away selection_changed signal

We get certain cases, in particular with SELECTION_MULTIPLE, where we
cannot figure out in advance of real_set_cursor() if the selection will
actually change.

tests: Add select/unselect calls

treeview: Add a CURSOR_INVALID flag to set_cursor_row()

This is necessary so the code doesn't attempt to do things with the
previous cursor, which currently cause crashes.

treeview: Replace lots of boolean args with flags

I want to add more and 3 boolean args is essentially unreadable code.

tests: Add sanity checks for selection

We want to know that the selection emits "changed" when it changes.

docs: Update code demo for newer glib

tests: Dump the operation that is performed to stderr

tests: Make treechanging test run as fast as possible

treeview: Handle the case where the cursor row gets deleted

Previously, the cursor would just become invalid, which used to
reselect the first row in the treeview later on (without a
cursor-changed signal). This leads to a crash now with the recent
refactorings.

The patch is longer than I'd like it to be, but the situation is kinda
complicated, because we want to make sure to move the cursor to a good
row. It works like this:
1) From the deleted row, search forward for the first row that is not
   going to be deleted and is not a separator.
2) If no such row exists, search backwards for a row that is not a
   separator.
3) If no such node exists, clear the cursor.

treeview: Replace cursor handling by keeping the node

Previously the code used a GtkTreeRowReference, which was (a) less
performant and more importantly (b) hiding errors.
The errors being hidden were the referenced row becoming invalid or
collapsed, because such rows would not be valid cursor rows and it would
be necesary to select a new row and emit a "cursor-changed" signal.

So if a crash bisects down to this commit, it is very likely that the
cursor has not correctly been updated and the cursor row is invalid.

tests: Put random number into local variable

This way, we can see what function had previously been called when the
checks fail.

treeview: Use _gtk_rbtree_contains() in a11y code

rbtree: Add _gtk_rbtree_contains()

It's useful in a bunch of places, so split it out

Fix move-resize emulation with XI2

https://bugzilla.gnome.org/show_bug.cgi?id=650943

wayland: Load cursors from wayland directory under XDG_DATA_DIRS

This means we don't have to have hardcoded "/usr/share/wayland" to find the
cursors.

This change also fixes up the warning messages for when loading fails.

Updated Breton translation

Updated Breton translation

Revert 3fd7dbc94074826755da7fc3237e439ba6e81ada

Calling gdk_keymap_add_virtual_modifiers causes _all_ virtual
modifiers to be added, which causes problem when they are co-located
on the same real modifier (as Super and Hyper often are). Effectively,
this made it impossible to enter key combinations involving Super,
since they all turn into Super+Hyper.

Test some virtual modifiers

This adds some accelerators involving Super and Hyper to testmerge.
This helps in testing our accel matching for those.

print to file: fixed lost filename in gui

Stop widget creating an event before updating both directory and
filename values.

https://bugzilla.gnome.org/show_bug.cgi?id=603823

overlay: Properly allocate the child upon addition

This ensures that after addition the child gets assigned the proper
allocation and the child window is properly shown/hidden.

overlay: Move code

Move code for child allocation into the child allocation function. Don't
keep it in the overlay allocation code.

See the next commit for why this is useful.

overlay: Create child window before setting parent

Otherwise, the window will not be used when set_parent() maps the child.

progressbar: Set state flags on draw

We were missing setting state flags on the style context before drawing.

widget: Unset window-unfocused in gtk_widget_unparent()

Widgets without a parent aren't inside a toplevel window so we must remove
window-unfocused as it doesn't make sense outside a toplevel.

https://bugzilla.gnome.org/show_bug.cgi?id=661428

treeview: Fix crash when scrolling to end

This broke in a4630d0e7b233479825b059e0df0e6d65b0e6734

https://bugzilla.gnome.org/show_bug.cgi?id=665741

window: copy back window surface data when the implicit paint is flushed

When an implicit paint is flushed during expose, e.g. because a
non-double buffered widget is painting, make sure to copy the existing
data from the window surface we rendered before flushing back to the
paint surface, instead of using an empty base.
Code was already handling that (and said so in the comment), but only
when no implicit paint was used at all, and not in the case when it's
flushed mid-expose.

window: trivial cleanup

window: remove unused arguments and variables

Trivial cleanup, no functional change.

modifierstyle: Fix compile errors

Introduced in 7bd2446801ffd0fc2ac5cc98fcd675ac4f3935d6

Fix demo to use a "normal" transparent value now that it works.

The last commit fixed this, so lets have a sane value in the demo code.

Remove broken optimization in style modifier overrides

We used to only set the override color or font if it was different
than the existing value. However, that means you can't change it to
an override that is the same as the default value for the property.

With this fixed you can e.g. override with a color of 0,0,0,0 which
you couldn't before.

Try to fix linking

We need to link against gdk against pangoft2 on X11.
Also add Requires.private to gdk-3.0.pc, and try to clean
things up some.

https://bugzilla.gnome.org/show_bug.cgi?id=665326

gtk-demo: Add test of transparent GdkWindows

This shows a semi-transparent shadow from a GtkOverlay window

Remove an unused variable

gtk,notebook: Invalidate the gap side when reordering tabs

This narrow area is invalidated so the gap follows the tab
being reordered.

notebook: Make the tab reordering dnd window transparent

statusicon: fix the build on !X11

Uninevitable fallout from my last patch to GtkStatusIcon. Thanks to Rob
Bradford for testing this.

widget: Update Pango context on state change

The layout depends on the state flags, so it must be updated when they
change.

label: Clear layout on state change

Selected or active text might be rendered differently.

label: Move callback to new stuff

We want to react to state flags changing, not to the (deprecated) state
type changing.

wayland: Port from wl_shell to wl_surface_shell

Previously all the commands that acted on the shell took the surface that was
to be acted on as parameter. Now we retrieve an object from the shell that
represents its state for the surface. With that wl_shell_surface object we can
then call methods on that.

Signed-off-by: Kristian Høgsberg <krh@bitplanet.net>

stylecontext: always add the ARROW style class when rendering an arrow

When gtk_render_arrow() is called, always add an ARROW style class to
the GtkStyleContext before rendering, so themes can specify a different
color for it.

https://bugzilla.gnome.org/show_bug.cgi?id=665420

stylecontext: add GTK_STYLE_CLASS_ARROW

For e.g. submenu items arrow in GtkMenuItems.

https://bugzilla.gnome.org/show_bug.cgi?id=665420

statusicon: port to GtkIconHelper

Just always render the pixbuf ourselves and set it on the GtkImage in
the X11 case.
Code for other backends was already rendering the pixbuf manually before
translating it into a native type, so this greatly simplifies the code.

https://bugzilla.gnome.org/show_bug.cgi?id=665409

Merge branch 'transparent-windows'

Make window handle transparent

Instead of painting the window background on the grip_window we now
only paint it on the GtkWindow->window, and we make the grip_window
have a transparent background.

We can't really make transparent window handle background optional
via css atm, because the handle color is actually based on the
background color, so if that is set to transparent we won't draw
anything.

gdk: gdk_window_get_update_area don't remove alpha covered areas

gdk_window_get_update_area is supposed to get the area where things
need painting, and remove them from the update areas. However, if
some area is covered by other windows with an alpha background we
can't just expect whatever the app choses to render in the update
area as correct, so we don't actually remove these areas, meaning
they will get correctly rendered when we get to the expose handlers.

gdk: gdk_window_move_region

gdk_window_move_region doesn't move children, so we can't copy
transparent child window regions with copyarea, so we remove these
from the copy region.

gdk: Fix alpha window background paintings on non-implicit paints

gdk: Remove unused _gdk_window_calculate_full_clip_region

Handle has_alpha_background for parent-relative backgrounds

entry: avoid a mismatched cairo_save/restore

Leftover from GtkIconHelper migration.

Remove unused alignment from GtkTooltip.

Plug tiny leak in the css parser

wayland: Update to recent API changes

GtkIconView: fix a memory corruption in the drag-n-drop code

You shall free with g_slice_free() what you allocate with
g_slice_new().

https://bugzilla.gnome.org/show_bug.cgi?id=665338

gdk: Fix repaint of layered region during move_region

gdk: Fix repaint of layered region during scroll

gdk: Fix repaint of layered region during raise

gdk: Make sure we don't copy to/from layered regions when moving a window

gdk: Track the layered area

We track the areas that have alpha coverage so that we can
avoid using these as sources when copying window contents.
We also don't remove such areas from the clipping regions so
that they are painted both by parent and child.

gdk: Track wether windows have alpha in the background

This will let us handle such windows differently in the
drawing machinery

testwindows: Add a button to cause a repaint

This makes it easy to find various kinds of repaint bugs

testwindows: Test transparent backgrounds

gdk: Expose bottommost windows first

This cleans up the expose handling a bit by using the existing
clip regions, and it allows us later to use painters algorithm
to do transparent windows.

gdk: Remove now unused region tags completely

image: use the default icon size if no icon size is provided

If no icon size is provided (e.g. when setting icon-name or stock-id
using the GObject property directly), use the default icon size. This
matches the previous GtkImage behavior.

image: minor cleanup

Introduce gtk_render_insertion_cursor

The new function provides an API that takes the PangoLayout and index
as input params, this way it handles strong and weak cursors internally
factoring out all code duplicated in the widgets that need to render
cursors.

https://bugzilla.gnome.org/show_bug.cgi?id=640317

Use the split-cursor setting for textview cursors

Explicitely check the split-cursor setting when drawing the textview
insertion cursor instead of relying on the cursor_direction set in the
textlayout. This makes the cursor drawin code more uniform with other
widgets in preparation to refactoring the cursor drawing code in a
shared function.

https://bugzilla.gnome.org/show_bug.cgi?id=640317

Rework GtkTextView cursor code.

Move the handling of primary/secondary cursors to gtktextdisplay, which
makes code simpler and more consistent to how GtkLabel and GtkEntry
draw cursors, which is useful in preparation to further refactoring.

https://bugzilla.gnome.org/show_bug.cgi?id=640317

Tiny cleanup.

https://bugzilla.gnome.org/show_bug.cgi?id=640317

Remove draw_insertion_cursor in label and entry

Remove the the draw_insertion_cursor wrapper which just converts from
GtkTextDir to PangoDirection

https://bugzilla.gnome.org/show_bug.cgi?id=640317

image: simplify code

We can unconditionally call into the icon helper methods now that we
removed the g_return_val_if_fail() checks.

entry: simplify code

We can unconditionally call into the icon helper methods now that we
removed the g_return_val_if_fail() checks.

icon-helper: remove useless checks

When we change storage type, we always clear the rest of the object, so
these checks do not add any safety.

icon-helper: don't warn out when the icon size is GTK_ICON_SIZE_INVALID

We can call ensure_icon_size() for an empty image; don't warn out in
that case, but just return a zero width.

icon-helper: don't reset pixel_size when clearing the helper

Don't reset the pixel size when clearing the helper; pixel-size is e.g a
property of GtkImage, which is stable unless explicitly set.

entry: always prepare the style context when using it for entry icons

Make sure to always prepare the entry's style context for the entry icon
style when passing it to GtkIconHelper.

cellrendererpix: make sure to select the default helper as a fallback

If we don't have expander pixbufs, select the default helper to render
the icon, even if the cell is an expander. This mimics what the old code
did.

cellrendererpix: port to GtkIconHelper

entry: port to GtkIconHelper

dnd: port to GtkIconHelper

image: port to GtkIconHelper

Use newly introduced GtkIconHelper to render pixbufs in GtkImage.

icon-helper: add GtkIconHelper private object

GtkIconHelper is a helper object to easily obtain a pixbuf from
different icon sources (e.g. a GIcon, an icon name, a stock id, ...).

Code is ported from GtkImage, which will be adapted in the next commit.

GtkCssProvider: don't segfault when CSS file is not found

 * Fix null dereference in gtk_css_provider_take_error()
 * And another in gtk_css_provider_emit_error()

https://bugzilla.gnome.org/show_bug.cgi?id=664537

XI2: Push error traps around XIQueryDevice calls

This may help against the crashes at resume time that some
people have been hitting when input devices mysteriously
disappear.

[GtkFileChooserDefault] Don't destroy and re-create the filename entry

This is a leftover from the big rework to remove the expanded/collapsed mode
in Save mode.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

[GtkPathBar] Remove unused error argument

It used to be that _gtk_path_bar_set_file() would return an error if
it wasn't able to switch to the specified file, but that hasn't been
the case for a long while now, since the file chooser became async.

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Avoid shadowing a variable

Move bg_color variable to an inner scope.

[path-bar] Add some comments to explain the async recursion on an I/O callback

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

[path-bar] Only reset the children's styles if the children actually changed

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

Remove unused argument

Signed-off-by: Federico Mena Quintero <federico@gnome.org>

filechooserdefault: Don't unref value twice

Double unref was introduced in eb02dacb370e8b4e6ef62328d0c09673b1234b32

https://bugzilla.gnome.org/show_bug.cgi?id=646461
https://bugzilla.gnome.org/show_bug.cgi?id=664137

treeview: Properly change cursor when row is collapsed

Previously, when the cursor was a descendant of the collapsed row,
the cursor path was set to the collapsed row, but this was not
communicated via cursor-changed events.

treeview: Reorder code a bit

This is mainly in preparation for the next commit, but also to reorder
the code to
1) prepare modification of tree
2) modify tree
3) emit signals

tests: Add new test that is currently failing